xen/arm: vgic: Drop iactive, ipend, pendsgi field
authorJulien Grall <julien.grall@linaro.org>
Mon, 16 Feb 2015 14:50:54 +0000 (14:50 +0000)
committerIan Campbell <ian.campbell@citrix.com>
Thu, 19 Feb 2015 16:56:42 +0000 (16:56 +0000)
commita41d809b0063c564224a08af11030fee8535dd37
tree408b6cc402d421f44c249d3bb96ecc5dff6871b3
parent10af92d1750fc37566876bcc7c95be7159e3277a
xen/arm: vgic: Drop iactive, ipend, pendsgi field

The current VGIC code doesn't support to change the pending and active status
of an IRQ via the (re-)distributor.

Futhermore, all the access size wasn't support correctly and some
registers was implemented as write-ignore. The latter make very
difficult for a kernel developer to find that we don't support R/W to
those registers.

Make the support consistent:
    - read will return 0 (RAZ)
    - write will print an error and inject a data abort to the guest

Also, those fields was never set and field such as ipend and pendsgi was
doing the same jobs.

Rather than wasting memory, we should better drop it. We could re-introduce
them if we need it when the support will be made.

Signed-off-by: Julien Grall <julien.grall@linaro.org>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
[ ijc -- s/XENLOG_ERR/XENLOG_G_ERR/g ]
xen/arch/arm/vgic-v2.c
xen/arch/arm/vgic-v3.c
xen/include/asm-arm/vgic.h